Conversation
Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
|
Not sure if this is the best place to put it, but it feels like the simplest solution, and it's right next to |
Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
|
I've also added a test: 90ed0db |
| default_level = (int)log_levels->default_logger_level; | ||
| rcutils_logging_set_default_logger_level(default_level); | ||
| if (RCUTILS_LOG_SEVERITY_UNSET != default_level) { | ||
| rmw_ret_t rmw_status = rmw_set_log_severity((rmw_log_severity_t)default_level); |
There was a problem hiding this comment.
What you are doing here makes a certain sense; if we are changing the default logging level, we should change it everywhere.
On the other hand, I'm not sure what semantic we really want --log-level to have. As it stands right now, the semantic is more-or-less "core ROS 2 code". If we add this code in, it becomes "core ROS 2 code + RMW backend code". I'm not entirely sure that's what we want to do.
I think this would be helped by spelling out what the log-levels mean in a more rigorous way, probably as part of ros2/design#314 .
There was a problem hiding this comment.
I understand. The fact that rmw implementations don't support the "unset" logging level/state definitely highlights this.
Are there any plans to tackle that soon-ish after Galactic? I can't commit to doing ros2/design#314, but I can probably help, and I can certainly implement whatever is in the design document for this.
There was a problem hiding this comment.
Are there any plans to tackle that soon-ish after Galactic?
The honest answer is that I don't know right now. We've been concentrating on Galactic, so we don't have a roadmap for H-Turtle yet. I do hope it is something that the ROS 2 community gets to for H-Turtle, as I think it is important.
There was a problem hiding this comment.
Got it. I'll see what I can do (in a few weeks) then.
I'll leave this draft here for now.
|
I'd be interested in resurrecting the effort here, especially to address ros2/rmw_zenoh#423. I'm in favor of having |
4 participants
Closes #915
This makes
rclcallrmw_set_log_severityif the default level is set, e.g. with--ros-args --log-level debug.Signed-off-by: Christophe Bedard bedard.christophe@gmail.com